pomera-ai-commander 1.2.7 → 1.2.9

package/bin/pomera-create-shortcut.js

@@ -0,0 +1,51 @@
+#!/usr/bin/env node
+
+/**
+ * Pomera AI Commander - Desktop Shortcut Creator
+ * 
+ * This script creates a desktop shortcut for the Pomera GUI.
+ */
+
+const { spawn } = require('child_process');
+const path = require('path');
+
+// Get the path to create_shortcut.py
+const shortcutScript = path.join(__dirname, '..', 'create_shortcut.py');
+
+// Find Python executable
+function findPython() {
+    const { execSync } = require('child_process');
+
+    try {
+        execSync('python3 --version', { stdio: 'ignore' });
+        return 'python3';
+    } catch (e) {
+        try {
+            execSync('python --version', { stdio: 'ignore' });
+            return 'python';
+        } catch (e) {
+            console.error('Error: Python is not installed or not in PATH');
+            process.exit(1);
+        }
+    }
+}
+
+const pythonCmd = findPython();
+
+// Get command line arguments
+const args = process.argv.slice(2);
+
+// Spawn the Python script
+const proc = spawn(pythonCmd, [shortcutScript, ...args], {
+    stdio: 'inherit',
+    cwd: path.join(__dirname, '..')
+});
+
+proc.on('close', (code) => {
+    process.exit(code || 0);
+});
+
+proc.on('error', (err) => {
+    console.error('Failed to create shortcut:', err.message);
+    process.exit(1);
+});

package/bin/pomera.js

@@ -0,0 +1,68 @@
+#!/usr/bin/env node
+
+/**
+ * Pomera AI Commander - GUI launcher
+ * 
+ * This script launches the Pomera GUI application.
+ */
+
+const { spawn } = require('child_process');
+const path = require('path');
+
+// Get the path to pomera.py
+const pomeraPath = path.join(__dirname, '..', 'pomera.py');
+
+// Find Python executable
+function findPython() {
+    const { execSync } = require('child_process');
+
+    // Try pythonw first (Windows - no console)
+    if (process.platform === 'win32') {
+        try {
+            execSync('pythonw --version', { stdio: 'ignore' });
+            return 'pythonw';
+        } catch (e) {
+            // Fall through
+        }
+    }
+
+    // Try python3 (Linux/macOS)
+    try {
+        execSync('python3 --version', { stdio: 'ignore' });
+        return 'python3';
+    } catch (e) {
+        // Fall back to python
+        try {
+            execSync('python --version', { stdio: 'ignore' });
+            return 'python';
+        } catch (e) {
+            console.error('Error: Python is not installed or not in PATH');
+            console.error('Please install Python 3.8 or higher');
+            process.exit(1);
+        }
+    }
+}
+
+const pythonCmd = findPython();
+
+// Spawn the Python GUI
+const app = spawn(pythonCmd, [pomeraPath], {
+    stdio: 'inherit',
+    cwd: path.join(__dirname, '..'),
+    detached: process.platform !== 'win32' // Detach on non-Windows
+});
+
+// Handle process exit
+app.on('close', (code) => {
+    process.exit(code || 0);
+});
+
+// Handle errors
+app.on('error', (err) => {
+    console.error('Failed to start Pomera:', err.message);
+    process.exit(1);
+});
+
+// Forward signals
+process.on('SIGINT', () => app.kill('SIGINT'));
+process.on('SIGTERM', () => app.kill('SIGTERM'));

package/core/database_schema.py

@@ -398,6 +398,21 @@ class DataTypeConverter:
         """
         import json
         
+        # Handle None or empty string for all types
+        if value_str is None or value_str == '':
+            if data_type in ('json', 'array'):
+                return [] if data_type == 'array' else {}
+            elif data_type == 'str':
+                return ''
+            elif data_type == 'int':
+                return 0
+            elif data_type == 'float':
+                return 0.0
+            elif data_type == 'bool':
+                return False
+            else:
+                return ''
+        
         if data_type == 'str':
             return value_str
         elif data_type == 'int':
@@ -407,6 +422,14 @@ class DataTypeConverter:
         elif data_type == 'bool':
             return value_str == '1'
         elif data_type in ('json', 'array'):
-            return json.loads(value_str)
+            # Handle whitespace-only strings as empty
+            if not value_str.strip():
+                return [] if data_type == 'array' else {}
+            try:
+                return json.loads(value_str)
+            except json.JSONDecodeError as e:
+                # Debug: print what value caused the error
+                print(f"DEBUG: JSON parse failed for data_type='{data_type}': value_str='{value_str[:100]}...' error={e}")
+                return [] if data_type == 'array' else {}
         else:
             return value_str  # Fallback to string

package/core/database_schema_manager.py

@@ -308,8 +308,10 @@ class DatabaseSchemaManager:
             for index_sql in table_indexes:
                 # Extract index name from CREATE INDEX statement
                 parts = index_sql.split()
-                if len(parts) >= 5 and parts[0].upper() == "CREATE" and parts[1].upper() == "INDEX":
-                    index_name = parts[4]  # "CREATE INDEX IF NOT EXISTS index_name"
+                # "CREATE INDEX IF NOT EXISTS idx_name ON table(...)"
+                # parts: [0]=CREATE [1]=INDEX [2]=IF [3]=NOT [4]=EXISTS [5]=idx_name
+                if len(parts) >= 6 and parts[0].upper() == "CREATE" and parts[1].upper() == "INDEX":
+                    index_name = parts[5]  # After "CREATE INDEX IF NOT EXISTS"
                     expected_indexes.add(index_name)
         
         # Get existing indexes

package/core/database_settings_manager.py

@@ -143,6 +143,27 @@ class NestedSettingsProxy:
         """Return a copy of the underlying data as a regular dictionary."""
         return self._data.copy()
     
+    def pop(self, key: str, *args):
+        """Remove and return a value from the nested settings.
+        
+        Args:
+            key: Key to remove
+            *args: Optional default value if key not found
+            
+        Returns:
+            The removed value, or default if provided and key not found
+        """
+        if args:
+            result = self._data.pop(key, args[0])
+        else:
+            result = self._data.pop(key)
+        
+        # Save the change to database
+        full_path = f"{self._parent_key}"
+        self._settings_manager.set_setting(full_path, self._data)
+        
+        return result
+    
     def _update_nested_value(self, data: Dict[str, Any], path: str, value: Any) -> None:
         """Update value in nested dictionary using dot notation."""
         keys = path.split('.')
@@ -1112,8 +1133,10 @@ class DatabaseSettingsManager:
                 
                 critical_issues = [i for i in issues if i.severity == 'critical']
                 if critical_issues:
-                    self.logger.error(f"Imported settings have {len(critical_issues)} critical issues")
-                    return False
+                    # Log specific issues for debugging but allow import to proceed
+                    for issue in critical_issues:
+                        self.logger.warning(f"Import validation issue: {issue.location} - {issue.message}")
+                    self.logger.warning(f"Imported settings have {len(critical_issues)} validation issues - proceeding anyway")
             
             # Save imported settings
             success = self.save_settings(settings_data)

package/core/dialog_manager.py

@@ -283,8 +283,8 @@ class DialogManager:
             try:
                 dialog_settings = self.settings_manager.get_setting("dialog_settings", {})
                 
-                # Validate settings structure
-                if not isinstance(dialog_settings, dict):
+                # Validate settings structure - accept dict or dict-like objects (NestedSettingsProxy)
+                if not isinstance(dialog_settings, dict) and not hasattr(dialog_settings, 'get'):
                     self.logger.warning(f"Invalid dialog_settings structure: {type(dialog_settings)}, using defaults")
                     return True
                 
@@ -663,8 +663,8 @@ class DialogManager:
             # Force a fresh read of dialog settings
             dialog_settings = self.settings_manager.get_setting("dialog_settings", {})
             
-            # Validate settings structure
-            if not isinstance(dialog_settings, dict):
+            # Validate settings structure - accept dict or dict-like objects (NestedSettingsProxy)
+            if not isinstance(dialog_settings, dict) and not hasattr(dialog_settings, 'get'):
                 self.logger.error(f"Invalid dialog_settings structure: {type(dialog_settings)}, skipping refresh")
                 return
                 

package/core/efficient_line_numbers.py

@@ -9,6 +9,7 @@ from tkinter import scrolledtext
 import platform
 import time
 import threading
+import hashlib
 from typing import Dict, List, Tuple, Optional, Any
 from dataclasses import dataclass
 
@@ -31,7 +32,7 @@ class EfficientLineNumbers(tk.Frame):
         
         # Configuration
         self.line_number_width = 50  # Adjustable width
-        self.debounce_delay = 5  # ms - very responsive updates
+        self.debounce_delay = 50  # ms - balanced responsiveness vs. efficiency
         self.cache_size_limit = 1000  # Maximum cached line positions
         
         # Create widgets
@@ -280,12 +281,12 @@ class EfficientLineNumbers(tk.Frame):
             return None
     
     def _get_content_hash(self) -> str:
-        """Get a hash of the current content for change detection."""
+        """Get a hash of the current content for change detection using MD5."""
         try:
             content = self.text.get("1.0", "end-1c")
-            # Simple hash based on content length and first/last chars
             if content:
-                return f"{len(content)}_{content[:10]}_{content[-10:]}"
+                # Use MD5 for reliable change detection (truncated for efficiency)
+                return hashlib.md5(content.encode('utf-8', errors='replace')).hexdigest()[:16]
             return "empty"
         except Exception:
             return "error"